services/horizon: expose date header #2316
Conversation
tomerweller
force-pushed
the
horizon-date-header
branch
from
5495aea to
6c52fa3
Compare
|
I've tested and rebased |
There was a problem hiding this comment.
🎉 I think it is good for us to give Horizon clients visibility into what time that Horizon says the data was accurate at, which is all this field will do accurately.
@tomerweller In #2315 you mentioned:
Horizon clients might be interested in obtaining an accurate timestamp.
I don't think clients should use Horizon as a time server which the statement could be interpreted to mean. Do we have any plans like that, or is the intended use just what I stated at the top?
|
The JS sdk already uses horizon server time but this functionality doesn't work inside of the browser due to I definitely don't think that horizon should be used as a time server but if a client is already communicating with a horizon server I don't find it offensive to use that value as a sanity test. |
closes #2315
do not merge yet